New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add filter and CRUD tests for AutoIncrement field #3415
Conversation
🦋 Changeset is good to goLatest commit: 5884503 We got this. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
It looks like we could use some more details. Good titles and descriptions help us fix things faster. If you can provide more information please update this issue. |
9c0ed65
to
7b85de7
Compare
api-tests/fields.test.js
Outdated
|
||
describe('Fields', () => { | ||
const testModules = globby.sync( | ||
[`packages/**/src/**/test-fixtures.js`, `!packages/fields-auto-increment/src/test-fixtures.js`], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't like having package-specific code in this general file. Can we find a way to implement this logic in such a way that it becomes part of the test-fixtures.js
configuration?
515a3b5
to
d60c7b7
Compare
2a4d218
to
228a8aa
Compare
9dde602
to
1898cad
Compare
6f87bcb
to
0026596
Compare
0026596
to
5884503
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
adapter
specific field API tests.fields.test.js
to insert items in an order as in array returned byinitItems()
function of the module.